Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(toc): affiche la table des matières au clique sur l'icone à gauche du titre #1090

Closed

Conversation

ggrossetie
Copy link
Collaborator

resolves #1088

Copy link

netlify bot commented Nov 7, 2024

Deploy Preview for stylo-docs canceled.

Name Link
🔨 Latest commit 9fbc251
🔍 Latest deploy log https://app.netlify.com/sites/stylo-docs/deploys/672cf09330c56400082c0c38

Copy link

netlify bot commented Nov 7, 2024

Deploy Preview for stylo-dev ready!

Name Link
🔨 Latest commit 9fbc251
🔍 Latest deploy log https://app.netlify.com/sites/stylo-dev/deploys/672cf0939f850f0008df5287
😎 Deploy Preview https://deploy-preview-1090--stylo-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@RochDLY
Copy link
Collaborator

RochDLY commented Nov 8, 2024

C'est super ! ça va grandement alléger le volet de gauche dans l'interface d'édition ! J'en profite pour réécrire ce dont nous avons discuté dans une conversation privée : en termes d'UX ça risque de ne pas être évident pour qui ne sait pas que la table des matières se cache derrière cet icône. Est-ce qu'on pourrait ajouter un effet sur l'overlay (au moins pour indiquer qu'il se passe un truc à cet endroit ?)

@RochDLY
Copy link
Collaborator

RochDLY commented Nov 8, 2024

On a aussi fait des tests avec une table des matières très longues et on se retrouve à scroller sous l'éditeur de texte (cf image ci-dessous). Je ne sais pas si ce comportement est souhaitable. Avec @ggrossetie nous proposons de tester de pouvoir scroller à l'intérieur de la fenêtre contenant la table des matières plutôt que dans toute l'interface.

image

@ggrossetie
Copy link
Collaborator Author

Suite à une discussion avec @maiwann et @RochDLY, on pense que la table des matières doit rester visible afin de permettre de naviguer rapidement. Autrement dit, on pense que le fait d'afficher la table des matières dans une info-bulle au dessus du texte (et qui se ferme dès qu'on clique sur un lien) n'est pas une bonne idée.

Je cloture 🚪

@ggrossetie ggrossetie closed this Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Afficher la table des matières de la même façon en édition collaborative comme en edition solo
2 participants